-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🐛 Fix missing redis-commaner password support #6131
🐛 Fix missing redis-commaner password support #6131
Conversation
… machines (⚠️ Devops) 🚨 (ITISFoundation#5923)
…tion#6052) Co-authored-by: Andrei Neagu <[email protected]>
Co-authored-by: Andrei Neagu <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Andrei Neagu <[email protected]>
Merged on behalf of @bisgaard-itis . Needed for staging release
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6131 +/- ##
=========================================
+ Coverage 84.5% 87.9% +3.4%
=========================================
Files 10 1401 +1391
Lines 214 58790 +58576
Branches 25 1480 +1455
=========================================
+ Hits 181 51733 +51552
- Misses 23 6746 +6723
- Partials 10 311 +301
Flags with carried forward coverage won't be shown. Click here to find out more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks
services/web/server/tests/unit/with_dbs/docker-compose-devel.yml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is it once REDIS_PASSWORD and in the other TEST_REDIS_PASSWORD??
Also minor but your commits are huge for just 2 files
Co-authored-by: Andrei Neagu <[email protected]>
The |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx.
TIP: Make sure you have a clean commit history in your PR. Note that you have 45 commits in the history for only two trivial file changes. :-)
|
What do these changes do?
When the password authentication for redis was added, I forgot to add the password to the redis-commander. This PR adds it.
Related issue/s
How to test
Dev-ops checklist